Skip to content

Fix dead SKILL.md path in test 23; add npm skill file guardrails (section 24) - #1

Draft
ProjectAI00 with Copilot wants to merge 2 commits into
mainfrom
copilot/add-dead-code-guardrails
Draft

Fix dead SKILL.md path in test 23; add npm skill file guardrails (section 24)#1
ProjectAI00 with Copilot wants to merge 2 commits into
mainfrom
copilot/add-dead-code-guardrails

Conversation

Copilot AI commented Mar 4, 2026 •
edited
Loading

Copy link
Copy Markdown

Section 23 of tests/integration.sh was silently failing because SKILL_SRC pointed to skills/imi/SKILL.md — a path that doesn't exist. The actual file lives at npm/skills/imi/SKILL.md. This is exactly the kind of dead-asset drift the codebase should guard against.

Changes

  • Fix SKILL_SRC path in section 23 — corrects the reference from skills/imi/SKILL.md to npm/skills/imi/SKILL.md, making the existing frontmatter and install-simulation checks functional again.

  • Add section 24: dead asset guardrails — loops over all six runtime-critical npm skill distribution files and fails immediately if any is missing:

NPM_SKILL_DIR="$(dirname "$0")/../npm/skills/imi"
for skill_file in SKILL.md ai-voice.md execute-mode.md ops-mode.md plan-mode.md imi.agent.md; do
  if [[ -f "$NPM_SKILL_DIR/$skill_file" ]]; then
    pass "npm skill file present: $skill_file"
  else
    fail "npm skill file MISSING: $skill_file" "$NPM_SKILL_DIR/$skill_file"
  fi
done

Any accidental deletion or rename of these files now surfaces immediately on make test.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/ProjectAI00/ai-db-imi/releases/latest
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3+json -H User-Agent: imi-cli REDACTED /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716ee�� /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716eef6.clap_derive.4aebc065d375203f-cgu.13.rcgu.o /home/REDACTED/work/ai-db-imi/ai-db-imi/target/release/deps/clap_derive-3a97accb5716eef6.clap_derive.4aebc065d375203f-cgu.14.rcgu.o (http block)
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3+json -H User-Agent: imi-cli REDACTED 39025a3fc5b904.b-s id_mmbuu8lk0doc ease/imi ent"�� stable-x86_64-unPOST stable-x86_64-un-H rm":"linux-x86_64","duration_ms":4,"is_ci":true,"interactive":false,"install_id":"iid_mmbuu8lk0 stable-x86_64-uncurl stable-x86_64-un-s "iid_mmbuu8lk0d stable-x86_64-un/dev/null (http block)
    • Triggering command: /usr/bin/curl curl -s --max-time 5 -H Accept: application/vnd.github.v3+json -H User-Agent: imi-cli REDACTED -H id_mmbuuytfl2ls curl ent"�� -o /dev/null :"linux-x86_64","duration_ms":1,"is_ci":true,"interactive":false,"install_id":"iid_mmbuuytfl2ls POST -H "iid_mmbuuytfl2 tr (http block)
  • us.i.posthog.com
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {"api_key":"phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF","event":"imi_init","distinct_id":"uid_mmbuu8lmkyo4lkvj","properties":{"version":"0.3.19","platform":"linux-x86_64","duration_ms":32,"is_ci":true,"interactive":false,"install_id":"iid_mmbuu8lk0do -C metadata=c54173db44b2d719 (dns block)
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {"api_key":"phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF","event":"imi_init","distinct_id":"uid_mmbuu8lmkyo4lkvj","properties":{"version":"0.3.19","platform":"linux-x86_64","duration_ms":2,"is_ci":true,"interactive":false,"install_id":"iid_mmbuu8lk0doc (dns block)
    • Triggering command: /usr/bin/curl curl -s -o /dev/null -X POST -H Content-Type: application/json -d {"api_key":"phc_exyd1ppU0ZS7McQ1ay1gxvCaUI2QfYPuMCTk4kawVKF","event":"imi_goal","distinct_id":"uid_mmbuu8lmkyo4lkvj","properties":{"version":"0.3.19","platform":"linux-x86_64","duration_ms":4,"is_ci":true,"interactive":false,"install_id":"iid_mmbuu8lk0doc (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Task: Add guardrails for future dead code drift

Description

Add lightweight checks or conventions that make dead-asset drift less likely, such as reference checks in tests/integration scripts or explicit ownership notes for runtime-critical folders. Keep this minimal and aligned with current project style.

Acceptance Criteria

At least one guardrail exists and is validated in existing test flow.

Relevant Files

  • tests/integration.sh
  • Makefile
  • npm/skills/imi/SKILL.md

Tools

  • (none)

Goal description

Audit and remove dead code paths, unused modules, stale scripts, and unnecessary folders that are no longer part of the active IMI runtime. Prioritize safety by proving non-usage before deletion, preserving required skill/runtime assets, and keeping behavior unchanged for supported flows. Include a final verification pass to ensure CLI commands and test suites still pass after removals.

Goal why

Reduce maintenance burden, confusion, and risk from stale assets while keeping the core loop reliable.

Prior work on this goal

  • (none)

Decisions affecting this goal

  • (none)

Workspace Path

/Users/aimar/Documents/Kitchen/imi/ai-db-imi

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@ProjectAI00
Add dead asset guardrail in tests/integration.sh (section 24)
aa3ac62 
Co-authored-by: ProjectAI00 <174952601+ProjectAI00@users.noreply.github.com>
Copilot AI changed the title [WIP] Add guardrails to prevent dead code drift Fix dead SKILL.md path in test 23; add npm skill file guardrails (section 24) Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ProjectAI00 ProjectAI00

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ProjectAI00